Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

fix(markdown): add margins to all markdown elements #986

Merged
merged 2 commits into from
Jan 16, 2024

Conversation

douglasbouttell-camunda
Copy link
Contributor

Closes https://github.com/camunda/product-hub/issues/2024

  • This PR adds a new form-js element or visually changes an existing component.

Screenshot

image

@CLAassistant
Copy link

CLAassistant commented Jan 15, 2024

CLA assistant check
All committers have signed the CLA.

Copy link
Contributor

@vsgoulart vsgoulart left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Code wise everything looks good

@douglasbouttell-camunda I just need to talk about the visual regression tests with you tomorrow

Copy link
Contributor

@Skaiir Skaiir left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good on my end, besides the visual regression stuff :)

Ah right actually @douglasbouttell-camunda you also need to sign the CLA when working on bpmn-io repos, shouldn't be a complex process. You just need to do it once.

@Skaiir
Copy link
Contributor

Skaiir commented Jan 16, 2024

grafik

Feel free to merge @douglasbouttell-camunda :)

@douglasbouttell-camunda douglasbouttell-camunda merged commit 5673041 into develop Jan 16, 2024
12 of 13 checks passed
@douglasbouttell-camunda douglasbouttell-camunda deleted the fix-markdown-spacing branch January 16, 2024 13:49
@bpmn-io-tasks bpmn-io-tasks bot removed the needs review Review pending label Jan 16, 2024
vsgoulart added a commit that referenced this pull request Jan 31, 2024
* chore: update actions/setup-node digest to d86ebcd (#967)

Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>

* fix(markdown): add margins to all markdown elements (#986)

* fix: improve image editor placeholder styles

Related to #994

* chore: adjust iframe placeholder font size

Related to #994

* feat: add `security` properties for iFrame

Closes #901

* feat: add warning step to iframe security attributes config

Related to #901

* fix: show iframe title in the editor

Closes #990

* chore(CHANGELOG): correct minor formatting issues (#1003)

* chore: update actions/cache action to v4 (#995)

Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>

* chore: update actions/upload-artifact digest to 26f96df (#979)

Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>

* fix: repeatable entry parameters now validate with errors

Closes #966

* feat: implement number field debouncing

Closes #958

* chore: remove superfluous useEffects

Related to #958

* fix: Prevent demo data from being created on edited tables

Closes #997

* chore: Add experimental tag

Related to #997

* chore: update actions/upload-artifact digest to d00351b (#1014)

Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>

* feat: implement HTML display component

Closes #1004

* feat(schema): support HTML component

Related to #1004

* chore: remove our dependency on preact-markup

Related to #1004

* fix: limit cm-editor properties height to 400px

Related to #1004

* chore: cleanup html and text components

Related to #1004

* fix: handle native CSS scoping & added tests

#1004

* chore: updated snapshots

* chore: Update deps (#1008)

* chore: Update deps

* chore: Revert properties panel update

* chore: update dependency source-map-loader to v5 (#988)

Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>

* chore: replace dependency npm-run-all with npm-run-all2 ^5.0.0 (#1012)

* chore: replace dependency npm-run-all with npm-run-all2 ^5.0.0

* chore: Update to 6.1.1

---------

Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>
Co-authored-by: Vinicius Goulart <[email protected]>

* fix: resize textarea on input changes (#1017)

Closes #1011

---------

Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>
Co-authored-by: bpmn-io-bot <[email protected]>
Co-authored-by: Douglas Bouttell <[email protected]>
Co-authored-by: Vinicius Goulart <[email protected]>
Co-authored-by: Niklas Kiefer <[email protected]>
Co-authored-by: Nico Rehwaldt <[email protected]>
Co-authored-by: Skaiir <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
spring cleaning Could be cleaned up one day
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants